Skip to content

Revise changelog for version updates and completion - #48

Merged
MstfSlm38 merged 1 commit into
26Q2.5from
MstfSlm38-patch-15
Nov 4, 2026
Merged

MstfSlm38 merged 1 commit into
26Q2.5from
MstfSlm38-patch-15

Conversation

@MstfSlm38

Copy link
Copy Markdown
Collaborator

Updated the changelog section to reflect the completion status of version 26Q3.5 and removed redundant entries.

Updated the changelog section to reflect the completion status of version 26Q3.5 and removed redundant entries.
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 4, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-04T14:32:19.370585Z d40bb7e PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

💡 Codex Review


P2 Badge Make the npm release test step runnable

When a release triggers this workflow, the build job stops at npm test: package.json defines no test script, and running the command here exits with “Missing script: test.” As a result, publish-npm never runs. Add a test script or remove this gate so package releases can proceed.


JSON.parse(rawData);

P2 Badge Validate backup shape before accepting imports

If a user selects syntactically valid JSON that is not the exported snippet array, such as {} or null, this check still returns success. The renderer then overwrites all_snippets_data with that value, and renderSnips fails when it calls .forEach, leaving the snippets unavailable. Validate the expected array and entry shape before returning success.


localStorage.setItem("codesnip_custom_background", String(reader.result));

P2 Badge Avoid storing large background images twice in localStorage

For images near the advertised 4 MiB limit, readAsDataURL produces roughly 5.3 MiB of base64 data. This stores it under codesnip_custom_background, then window.applySettings() also serializes the same value into app_settings (renderer.js:458-461), so accepted files can exceed Chromium's localStorage quota and fail to apply or persist. Store the image once or use storage suited to larger files.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@MstfSlm38
MstfSlm38 merged commit 6c66782 into 26Q2.5 Nov 4, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant